Skip to content

Determine missing license information for packages to be installed#1149

Merged
trz42 merged 127 commits intoEESSI:mainfrom
hvelab:licenses
Feb 23, 2026
Merged

Determine missing license information for packages to be installed#1149
trz42 merged 127 commits intoEESSI:mainfrom
hvelab:licenses

Conversation

@hvelab
Copy link
Contributor

@hvelab hvelab commented Aug 8, 2025

New PR as per EESSI/software-layer-scripts#55

Needs further testing and refinement, as soon as scripts are tested I will move them to software-layer-scripts

trz42
trz42 previously requested changes Feb 21, 2026
Copy link
Collaborator

@trz42 trz42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this PR adds a workflow that determines for which packages license information is missing and supplies a patch that can be used to add the missing information (manual step). At the moment it succeeds when license information is missing. We may want to change this in the future. To my understanding it does not verify if all packages (to be added) allow redistribution, but this was likely not the intention of the PR. Nevertheless it's an important step towards checking licenses and thus we should not delay start using this workflow.

Went through this and tested it in trz42#94 and trz42#95

I suggest to only include the changes that are used in trz42#95 and made a PR to yours (hvelab#7), so you can easily apply the changes.

Before we can merge this PR, the easystack file needs to be removed.

@trz42 trz42 changed the title Draft: Licensing Determine missing license information for packages to be installed Feb 23, 2026
@trz42 trz42 dismissed stale reviews from ocaisa and themself February 23, 2026 12:04

All resolved

Copy link
Collaborator

@trz42 trz42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @hvelab ! Great step towards license checking.

@trz42 trz42 merged commit e0c6143 into EESSI:main Feb 23, 2026
79 of 80 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants